Skip to content

Adds an upload definitions method to upload the definitions#220

Merged
michaelklishin merged 8 commits intomichaelklishin:masterfrom
shubhang93:master
Dec 12, 2021
Merged

Adds an upload definitions method to upload the definitions#220
michaelklishin merged 8 commits intomichaelklishin:masterfrom
shubhang93:master

Conversation

@shubhang93
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Owner

@michaelklishin michaelklishin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The newly added test fails because no arguments provided on the input data, which makes in fail validation on the HTTP API endpoint side.

Comment thread definitions.go Outdated
@shubhang93
Copy link
Copy Markdown
Contributor Author

All the tests are passing, can you merge the changes @michaelklishin

@shubhang93
Copy link
Copy Markdown
Contributor Author

also can we use a docker container, instead of installing rabbitmq-server on a debian OS?

@michaelklishin
Copy link
Copy Markdown
Owner

Containers make sense for Actions. I don't want to switch to container-based nodes only for local development but would consider a PR that adds support for containers as an option locally, similarly to how Bunny does it.

@shubhang93
Copy link
Copy Markdown
Contributor Author

This current PR adds support to use containers locally

@michaelklishin michaelklishin merged commit d3acd47 into michaelklishin:master Dec 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants